Skip to content

fix(appointments): Make scheduled appointments meeting conversations …#8127

Merged
SebastianKrupinski merged 1 commit intomainfrom
bugfix/noid/make-appointments-talk-meetings
Apr 6, 2026
Merged

fix(appointments): Make scheduled appointments meeting conversations …#8127
SebastianKrupinski merged 1 commit intomainfrom
bugfix/noid/make-appointments-talk-meetings

Conversation

@nickvergessen
Copy link
Copy Markdown
Member

@nickvergessen nickvergessen commented Apr 2, 2026

…in Talk

Steps

  1. Create an "Appointments schedule" with Talk conversation option
  2. Book a meeting and confirm it
Before After
Conversation is always visible Conversation is only visible when the meeting is close or list is filtered for meetings
Bildschirmfoto vom 2026-04-02 17-22-45 Bildschirmfoto vom 2026-04-02 17-23-07
Conversation persists after the call Conversation expires after 28 days
Bildschirmfoto vom 2026-04-02 17-25-32 Bildschirmfoto vom 2026-04-02 17-25-22

…in Talk

Signed-off-by: Joas Schilling <coding@schilljs.com>
Copy link
Copy Markdown
Contributor

@miaulalala miaulalala left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, not sure about the "public" option but it kinda makese sense

@nickvergessen
Copy link
Copy Markdown
Member Author

not sure about the "public" option but it kinda makese sense

That was there before, it's needed as the other user is a public link guest

@SebastianKrupinski SebastianKrupinski merged commit eaa56a0 into main Apr 6, 2026
41 checks passed
@SebastianKrupinski SebastianKrupinski deleted the bugfix/noid/make-appointments-talk-meetings branch April 6, 2026 13:42
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@nickvergessen
Copy link
Copy Markdown
Member Author

/backport to stable6.3

@backportbot backportbot bot added the backport-request A backport was requested for this pull request label Apr 7, 2026
@nickvergessen
Copy link
Copy Markdown
Member Author

Backporting as per #8139 (comment)

@nextcloud nextcloud deleted a comment from backportbot bot Apr 7, 2026
@nickvergessen
Copy link
Copy Markdown
Member Author

Nvm seems this was merged before branch off

@nickvergessen nickvergessen removed the backport-request A backport was requested for this pull request label Apr 7, 2026
@SebastianKrupinski
Copy link
Copy Markdown
Contributor

/backport to stable6.3

This was merged before branch off, should not be needed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants